Skip to content

feat(integrations): Send SLO lifecycle metrics to Sentry via SDK#112839

Merged
trevor-e merged 1 commit intomasterfrom
telkins/slo-sentry-metrics
Apr 13, 2026
Merged

feat(integrations): Send SLO lifecycle metrics to Sentry via SDK#112839
trevor-e merged 1 commit intomasterfrom
telkins/slo-sentry-metrics

Conversation

@trevor-e
Copy link
Copy Markdown
Member

Summary

  • Adds a direct sentry_sdk.metrics.count() call in EventLifecycle.record_event() so SLO metrics (started, success, halted, failure) are sent to Sentry's metrics product at 100% with integration_id included as a tag.
  • The existing metrics.incr() call (Datadog) is untouched.
  • This bypasses the DualWriteMetricsBackend experimental backend which only samples at 5%, giving us full-fidelity SLO data in Sentry without the cardinality cost on Datadog.

Companion PR in seer: https://github.com/getsentry/seer/pull/new/telkins/slo-sentry-metrics

Test plan

  • Verify metric appears in Sentry Metrics explorer as sentry.integrations.slo.started with integration_id tag
  • Confirm existing Datadog metrics are unaffected

Adds a direct sentry_sdk.metrics.count() call alongside the existing
metrics.incr() in record_event() so SLO metrics (started, success,
halted, failure) are sent to Sentry at 100% with integration_id,
bypassing the DualWrite experimental backend's 5% sample rate.
@trevor-e trevor-e requested review from a team as code owners April 13, 2026 19:31
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 13, 2026
@trevor-e trevor-e merged commit 0d2da2c into master Apr 13, 2026
74 checks passed
@trevor-e trevor-e deleted the telkins/slo-sentry-metrics branch April 13, 2026 20:04
@trevor-e trevor-e added the Trigger: Revert Add to a merged PR to revert it (skips CI) label Apr 15, 2026
@getsentry-bot
Copy link
Copy Markdown
Contributor

PR reverted: 7e3195c

getsentry-bot added a commit that referenced this pull request Apr 15, 2026
…SDK (#112839)"

This reverts commit 0d2da2c.

Co-authored-by: trevor-e <1447798+trevor-e@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components Trigger: Revert Add to a merged PR to revert it (skips CI)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants